Skip to content

chore: refactor build system to use PCRE2 #3383

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 17, 2025

Conversation

airween
Copy link
Member

@airween airween commented May 15, 2025

what

This PR refactors the build system and the relevant source files to make the PCRE2 library as default. If you wish to continue building with the legacy PCRE library, you must explicitly specify it using the following command:
configure --with-pcre. Without that the default will be PCRE2.

The PR also changes the GH worfklow build matrix.

why

Old PCRE library is deprecated, it's time to change to PCRE2.

references

See the same change in v3.

This PR is based on #3345, but that wasn't finished.

@airween airween requested review from theseion and fzipi May 15, 2025 20:02
Co-authored-by: Max Leske <[email protected]>
@airween airween requested a review from theseion May 16, 2025 07:40
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
46.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@airween airween added the 2.x Related to ModSecurity version 2.x label May 16, 2025
@airween airween merged commit f2996d6 into owasp-modsecurity:v2/master May 17, 2025
81 of 82 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Related to ModSecurity version 2.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants